lua: don't use | operator on VLC errors

This could end up with unknown/invalid error values.

(cherry picked from commit 586b28f4aab1f936d666bd3dab7904793c3012d7)

Gbp-Pq: Name 0056-lua-don-t-use-operator-on-VLC-errors.patch

Taglib: Use ID3v2Tag() instead of tag() for RIFF::WAV::File

`WriteMetaToId3v2` expects a `ID3v2::Tag` instead of `Tag`, but Since TagLib v2.0,
`RIFF::WAV::File::tag()` returns a `Tag` instead of `ID3v2::Tag`, hence replace
the usage of `tag()` method with `ID3v2Tag()`.

https://github.com/taglib/taglib/blob/master/taglib/riff/wav/wavfile.h#L124

Additionally, to resolve the compilation error, the function signatures of
`insert` and `removeBlock` have been adjusted to align with the base class
functions.

(cherry picked from commit ec29dfca1e59530dd412d779e0b045079b72ffb6)

Gbp-Pq: Name 0055-Taglib-Use-ID3v2Tag-instead-of-tag-for-RIFF-WAV-File.patch

taglib: remove VLC path define check

It is no longer needed with contribs taglib updated to 1.13.

(cherry picked from commit c5851fd495ed6de1d3712e64a2b1cc1e4e9059f4)

Gbp-Pq: Name 0054-taglib-remove-VLC-path-define-check.patch

taglib: don't cast the file st_size to a possibly smaller type

We can just use auto and always get the proper type.

(cherry picked from commit 7413126744e0b494856ba9acc5ae3c27ce4af625)

Gbp-Pq: Name 0053-taglib-don-t-cast-the-file-st_size-to-a-possibly-sma.patch

meta: taglib: use std::min

(cherry picked from commit 584fd17a9445d8d5f7d692b7416eec09b9240b0e)

Gbp-Pq: Name 0052-meta-taglib-use-std-min.patch

taglib: Fix invalid version check

(cherry picked from commit 53260ca5ae906f1e35791d14e4b588d25c04e51b)

Gbp-Pq: Name 0051-taglib-Fix-invalid-version-check.patch

taglib: Use nullptr instead of 0 for pointers

(cherry picked from commit 89e20ea0d87f414f763ce38ff5579aec61f71b42)

Gbp-Pq: Name 0050-taglib-Use-nullptr-instead-of-0-for-pointers.patch

taglib: Implement new StreamTypeResolver interface when available

refs #26602

(cherry picked from commit 268b5b8bc1f1109c4fc69b22e53095c6d81faa76) (rebased)

Gbp-Pq: Name 0049-taglib-Implement-new-StreamTypeResolver-interface-wh.patch

taglib: Remove explicit invocation of default constructor

This is already done implicitly

(cherry picked from commit 93842f650548d6f29ec13cf0679bf49cf5ce107c)

Gbp-Pq: Name 0048-taglib-Remove-explicit-invocation-of-default-constru.patch

taglib: Remove use of deprecated String::isNull

(cherry picked from commit c404fdb24183031a419667639846edddca3401f8) (edited)
edited:
- 294cbdbaee0209bc88ca7bec9164da8da4ac44ef was merged before this patch

Gbp-Pq: Name 0047-taglib-Remove-use-of-deprecated-String-isNull.patch

taglib: Remove usages of deprecated itemListMap

contains() was added in 1.10 and item() is supported since 1.9

(cherry picked from commit ac59d0ba59ba800c52c0a92ec1d9576d2653dbe2)

Gbp-Pq: Name 0046-taglib-Remove-usages-of-deprecated-itemListMap.patch

taglib: ByteVector::null is now deprecated

(cherry picked from commit d2663d6c3fe3af76bdefd58367b4a45c598b83e4)

Gbp-Pq: Name 0045-taglib-ByteVector-null-is-now-deprecated.patch

taglib: Don't convert base64 strings to UTF8

The convertion will incur a cost which we don't want to pay since the
art needs to be encoded in base64, which doesn't include anything
outside of ASCII

(cherry picked from commit 65dfe0cdc97bbb98ad34a690989636d40f326e32)

Gbp-Pq: Name 0044-taglib-Don-t-convert-base64-strings-to-UTF8.patch

taglib: fix variable shadowing

list is already a different local list defined earlier in the function.

(cherry picked from commit 34ca2a286edda76a9718cfa1e50bd3e4ea51702c)

Gbp-Pq: Name 0043-taglib-fix-variable-shadowing.patch

meta: taglib: reject non expected schemes

refs #25035

(cherry picked from commit 8191ac5727a3525a6bee99e1674167a7894f9ccb) (rebased)
rebased:
- 3.0 already has a26e2ba370d898dda04c718176af7d2e329dab7a above

Gbp-Pq: Name 0042-meta-taglib-reject-non-expected-schemes.patch

meta_engine: taglib: restrict unlimited reads on streams

taglib reads unlimited if no IDv3 or MPEG header is found at the
beginning.

(cherry picked from commit 8ec8b0d2add84b6d6b0c043e1c4f241998915ecd) (rebased)
rebased:
- only used with Taglib 1.11+ which is not always the case with 3.0

Gbp-Pq: Name 0041-meta_engine-taglib-restrict-unlimited-reads-on-strea.patch

meta_engine: taglib: add sequential read limit on VlcIostream

(cherry picked from commit d375284a144e6b18896deda3a86b9560bc04f3d8) (rebased)
rebased:
- 3.0 already has 1.11 ifdef

Gbp-Pq: Name 0040-meta_engine-taglib-add-sequential-read-limit-on-VlcI.patch

meta_engine: taglib: reject read on failed seek

(cherry picked from commit 837b583e4e4e531ff8b11f13300c7cbc565beb9f) (rebased)
rebased:
- 3.0 already has 1.11 ifdef

Gbp-Pq: Name 0039-meta_engine-taglib-reject-read-on-failed-seek.patch

meta_engine: taglib: increment VlcIostream readpos

(cherry picked from commit 0cbcf504b43a02ddb7ecc1c65c0c1e82f39b4f1c)

Gbp-Pq: Name 0038-meta_engine-taglib-increment-VlcIostream-readpos.patch

taglib: fix -Wdeprecated-copy warnings

Warnings were happening because we were using copy operator= as the
assigned object was already initialized. This patch use the copy
constructor or move constructor instead if available.

Those warnings were:

../../modules/meta_engine/taglib.cpp: In function ‘void ReadMetaFromXiph(TagLib::Ogg::XiphComment*, demux_meta_t*, vlc_meta_t*)’:
../../modules/meta_engine/taglib.cpp:672:39: warning: implicitly-declared ‘TagLib::StringList& TagLib::StringList::operator=(const TagLib::StringList&)’ is deprecated [-Wdeprecated-copy]
  672 |     list = tag->fieldListMap()[keyName];                                       \
      |                                       ^
../../modules/meta_engine/taglib.cpp:681:5: note: in expansion of macro ‘SET’
  681 |     SET( "COPYRIGHT", Copyright );
      |     ^~~
In file included from /usr/include/taglib/fileref.h:30,
                 from ../../modules/meta_engine/taglib.cpp:58:
/usr/include/taglib/tstringlist.h:59:5: note: because ‘TagLib::StringList’ has user-provided ‘TagLib::StringList::StringList(const TagLib::StringList&)’
   59 |     StringList(const StringList &l);
      |     ^~~~~~~~~~

Signed-off-by: Jean-Baptiste Kempf <jb@videolan.org>
(cherry picked from commit be4f907a0e834c10277b8217520f745238da1c13)

Gbp-Pq: Name 0037-taglib-fix-Wdeprecated-copy-warnings.patch

taglib: Enable prefetch/cache on the IOStream

We can't do it through vlc_stream_NewURL as it will include the
skiptags filter

(cherry picked from commit 976cd219465a72daa02d22e60c9ac4c704c0b18b)

Gbp-Pq: Name 0036-taglib-Enable-prefetch-cache-on-the-IOStream.patch

taglib: Always use IOStream to read the media

(cherry picked from commit 4bc7607f31f80264b5e61fbd187f2f8d2f3bb604) (edited)
edited:
- keep the support for taglib older than 1.11 which doesn't have the
  proper FileRef constructor to use it
- the aacresolver is only called for Taglib 1.11+ on 3.0
- 4.0 doesn't make a difference for UWP or regular Windows

Gbp-Pq: Name 0035-taglib-Always-use-IOStream-to-read-the-media.patch

taglib: IOStream: Return the url as fileName

Since the only part that matters to taglib is the filename, and more
specifically the extension part.

(cherry picked from commit 026a76066efa78f1934ab2c177669a2e6a519cff)

Gbp-Pq: Name 0034-taglib-IOStream-Return-the-url-as-fileName.patch

modules: taglib: simplify reading embedded art

(cherry picked from commit 3533f90eb44398abe6cff1b14ece757654449acf) (rebased)
rebased:
- 294cbdbaee0209bc88ca7bec9164da8da4ac44ef has been merged above

Gbp-Pq: Name 0033-modules-taglib-simplify-reading-embedded-art.patch

taglib: .m4v are already registered as MP4 files

https://github.com/taglib/taglib/pull/662
(cherry picked from commit 84962e474bfefa242527f7c3ddffe6ed4f697fbe) (rebased)

Gbp-Pq: Name 0032-taglib-.m4v-are-already-registered-as-MP4-files.patch

aom: make BGR matching stricter

This is equivalent to the fix in ece317a245e8c32a5efc11d5e82dcd8a28a2f10b
as the aom decoder had the same issue reported in #28776.

(cherry picked from commit 0001e85a098ee4e6e2526eaec8a76899c2b23b90)

Gbp-Pq: Name 0031-aom-make-BGR-matching-stricter.patch

packetizer: dts: check for frame_size for the first sync

Only substreams are allowed to have a frame_size of 0.

(cherry picked from commit 16b31aa57e3919dbe661c213868dee4b8bc1e3bd)
Signed-off-by: Thomas Guillem <thomas@gllm.fr>
Gbp-Pq: Name 0030-packetizer-dts-check-for-frame_size-for-the-first-sy.patch

dts_header: initialize SUBSTREAM_LBR size to 0

It's a substream and can't work alone, so a size of 0 is legitimate.

Fixes #28773 (Use of uninitialised value)

(cherry picked from commit b23f09b258dc63c2221af502ec856630f68817e5)
Signed-off-by: Thomas Guillem <thomas@gllm.fr>
Gbp-Pq: Name 0029-dts_header-initialize-SUBSTREAM_LBR-size-to-0.patch

dav1d: add dav1d-all-layers flag

This will output all spatial layers if requested, defaults to false (the built-in dav1d
behavior would defaults this to true).

Fix suggested-by Maryla Ustarroz

Fixes #28776

(cherry picked from commit 6ffede3f445586e5d368de936666cabb84f6a16f)

Gbp-Pq: Name 0028-dav1d-add-dav1d-all-layers-flag.patch

dav1d: treat RGB formats separately

This avoids incorrectly matching as GBR.
Fixes #28763 which was missing checks for the matrix and primaries.

This also replaces the loop with a constant-time lookup.

(cherry picked from commit ece317a245e8c32a5efc11d5e82dcd8a28a2f10b)

Gbp-Pq: Name 0027-dav1d-treat-RGB-formats-separately.patch

Use `vlc_fourcc_GetCodec` to map XVID to MP4V

Gbp-Pq: Name 0026-Use-vlc_fourcc_GetCodec-to-map-XVID-to-MP4V.patch

codec: videotoolbox: remove sw decoder fallback for mp4v

commit 7a2bf498d545d500d30636970fa930eb54de5569 insert packetizer for
xvid mpeg4 video, so now hw decoder can decode xvid encoded video
without any playback issues.

(cherry picked from commit a9343f68bb19a1c20e45e1029848326020b0c69e)

Gbp-Pq: Name 0025-codec-videotoolbox-remove-sw-decoder-fallback-for-mp.patch

demux: AVI insert packetizer for XVID MPEG-4 video

Older DivX-encoded videos commonly use an method called packed
bitstream which puts several video frames into a single AVI chunk.
Since Packed bitstream isn't standard MPEG-4 it causes playback issues
with some hw decoders.

(cherry picked from commit 980e4d7449e9e06bef525d9049dff5b9b5ecd673)

Gbp-Pq: Name 0024-demux-AVI-insert-packetizer-for-XVID-MPEG-4-video.patch

demux: asf: add missing subpayloads drain on EOF

refs #28716

(cherry picked from commit 6bf0a1407b5b9aef792ef94a57142254d079e8fd)

Gbp-Pq: Name 0023-demux-asf-add-missing-subpayloads-drain-on-EOF.patch

packetizer: hxxx: add missing drain for non annexb content

(cherry picked from commit 4f0a39ada1bd9314601e80d7f48855462524d1fc)

Gbp-Pq: Name 0022-packetizer-hxxx-add-missing-drain-for-non-annexb-con.patch

packetizer: hxxx: pass opaque to callbacks instead of decoder

(cherry picked from commit 09655dea51c0d383cc86c0f5020e3bea160c69bf)

Gbp-Pq: Name 0021-packetizer-hxxx-pass-opaque-to-callbacks-instead-of-.patch

packetizer: hxxx_common: rename function pointer

(cherry picked from commit 6e7b32f64db918b3aa2deefe885571b734d32f53)

Gbp-Pq: Name 0020-packetizer-hxxx_common-rename-function-pointer.patch

opencv: check the image conversion worked

Otherwise we release the output picture that would have been used and returned.

(cherry picked from commit 19813291fcfb31ddd91a2e4d0f1d37ad3239d300) (rebased)
rebased:
- p_filter->p_sys is p_sys on 4.0
Signed-off-by: Steve Lhomme <robux4@ycbcr.xyz>
Gbp-Pq: Name 0019-opencv-check-the-image-conversion-worked.patch

magnify: check the image conversion worked

return NULL on failure as we cannot produce the required output.

Fixes #28707

(cherry picked from commit 990de75bc0a7db5f2c4e4cb88e868b75b689ac7d) (rebased)
rebased:
- picture_CopyPixels is called picture_CopyVisiblePixels on 4.0
Signed-off-by: Steve Lhomme <robux4@ycbcr.xyz>
Gbp-Pq: Name 0018-magnify-check-the-image-conversion-worked.patch

spectrogram: allows better visualization of low frequencies

In some cases, Y which represents the height of a column was equal to 1
and therefore when passed through the log, the output displayed was
zero, by adding this 0.1, this allows you to see a column when y = 1
without really changing the height of each column

(cherry picked from commit c347fed91e76bd31387171e1ff67224c21194362)

Gbp-Pq: Name 0017-spectrogram-allows-better-visualization-of-low-frequ.patch

spectrogram: convert int16 to unsigned for correct range

p_dest is used to set the height of the column, and it is used
as a variable defined from zero to 2^16 - 1. It is therefore
considered in the rest of the program as an unsigned int 16.
Moreover, the value that we put inside are a sum of two squared
real value, so they are necessarily positive.

(cherry picked from commit 94a8d152eadd53073305c95d221eca7623b4ed6c)

Gbp-Pq: Name 0016-spectrogram-convert-int16-to-unsigned-for-correct-ra.patch

spectrogram: fix FFT result scaling factor

This function is just a scaling operation that passes a value between 0 and
FFT_SCALING_VALUE to a value between 0 and 2^16-1.

It is therefore a power that must be applied here and not a xor.

Moreover, the initial formula was quite wrong since it was assuming that the max_input
value, which is a signed int 16, was 2^15 = 32768 where it is in fact 2^15-1=32767.

Moreover, the initial formula wasn't taking into account that, since the
output of the fft_perform is the sum of two variables which have for max value
(FFT_BUFFER_SIZE / 2 * INT16_MAX)^2, then we need to multiply the final max
value by 2.

Corrects the following compiler warnings:
visualization/glspectrum.c:528:43: warning: result of ‘2^16’ is 18; did you mean ‘1 << 16’ (65536)? [-Wxor-used-as-pow]
visualization/visual/effects.c:216:40: warning: result of ‘2^16’ is 18; did you mean ‘1 << 16’ (65536)? [-Wxor-used-as-pow]

Fixes #28506

Co-authored-by: Disha Baghel <bagheldisha708@gmail.com>
(cherry picked from commit 77a08a60f72bda081d144119e4d0a57a9c3c72f6)

Gbp-Pq: Name 0015-spectrogram-fix-FFT-result-scaling-factor.patch

UPnP: remove SAT>IP channel list fallback

When introducing SAT>IP support, we added a fallback mechanism on
officially published channel lists in case the user neither specified
a custom list nor the used set-top box provided one.

With the end of the SAT>IP Alliance that was dissolved 3 years ago, the
fallback server is no longer available and there will be no replacement
so this feature was removed.

Fixes #28684

(cherry picked from commit 3df7b6e1187030591febd8e1bdd2712c0d6af132)

Gbp-Pq: Name 0014-UPnP-remove-SAT-IP-channel-list-fallback.patch

demux: adaptive: fix start with indexes only

refs #28683

(cherry picked from commit 9f9c466bc14c1dcd75564c7cd18d031ea9e76325)

Gbp-Pq: Name 0013-demux-adaptive-fix-start-with-indexes-only.patch

packetizer: h264: fix reading prediction weight tables

(cherry picked from commit f878115e184a4d896e4df39bccbde32ea6d9357c)

Gbp-Pq: Name 0012-packetizer-h264-fix-reading-prediction-weight-tables.patch

qt: hardcode the name of the shortcut for the AMD VQ Enhancer

Otherwise the name is the same of the DLL which doesn't correspond to
the shortcut we need to use.

Fixes #28691

Gbp-Pq: Name 0011-qt-hardcode-the-name-of-the-shortcut-for-the-AMD-VQ-.patch

sftp: fix seek for large files on 32-bit OS

(cherry picked from commit 4698e0336447de8dc6f7214743082cc55b6ebd77)

Gbp-Pq: Name 0010-sftp-fix-seek-for-large-files-on-32-bit-OS.patch

transcode: fix picture fifo leak

Draining sets the abort flag. It was skipping the picture fifo deletion
before.

Gbp-Pq: Name 0009-transcode-fix-picture-fifo-leak.patch

transcode: fix destructive assignment after drain

The drain checks are done after the first encoder output fetch. At this
point, `out` is already filled with some frames gotten from the encoder
some lines above:

```
// ...
    if( p_sys->i_threads >= 1 )
    {
        /* Pick up any return data the encoder thread wants to output. */
        vlc_mutex_lock( &p_sys->lock_out );
        *out = p_sys->p_buffers;
        p_sys->p_buffers = NULL;
        vlc_mutex_unlock( &p_sys->lock_out );
    }

// ...
```

This assignment currently leaks all previously gathered frames to
replace them by the drained output. This patch appends the drained
frames to the existing output instead.

Gbp-Pq: Name 0008-transcode-fix-destructive-assignment-after-drain.patch

vlc_common: remove warnings on VLC_OBJECT

Fix an infinite amount of warnings on Darwin:

    ../../include/vlc_variables.h:563:5: warning: due to lvalue conversion of the controlling expression, association of type 'const struct vlc_common_members' will never be selected because it is qualified [-Wunreachable-code-generic-assoc]
        var_Create( p_obj, psz_name, VLC_VAR_STRING | VLC_VAR_DOINHERIT
        ^
    ../../include/vlc_variables.h:122:39: note: expanded from macro 'var_Create'
    #define var_Create(a,b,c) var_Create( VLC_OBJECT(a), b, c )
                                          ^
    ../../include/vlc_common.h:481:15: note: expanded from macro 'VLC_OBJECT'
            const struct vlc_common_members: (const vlc_object_t *)(&(x)->obj) \

Gbp-Pq: Name 0007-vlc_common-remove-warnings-on-VLC_OBJECT.patch

opus_header: fix channel mapping family 1 parsing

Fixes #27808

(cherry picked from commit 79fa6af0a98921f9d34933761f4fe20ef6c35309)

Gbp-Pq: Name 0006-opus_header-fix-channel-mapping-family-1-parsing.patch

demux: avi: do not set up invalid bitsperpixel

UINT16_MAX is set and propagated from the bitmap header reader

refs #28661

Gbp-Pq: Name 0005-demux-avi-do-not-set-up-invalid-bitsperpixel.patch

demux: image: detect ICC profile before JFIF data

Fixes #18857

(cherry picked from commit 5ffd36ffa26a83ae498373f7d0ace1d82ab952f4)
Signed-off-by: Steve Lhomme <robux4@ycbcr.xyz>
Gbp-Pq: Name 0004-demux-image-detect-ICC-profile-before-JFIF-data.patch

Bump module ABI for time_t transition

Gbp-Pq: Name 0003-Bump-module-ABI-for-time_t-transition.patch

Do not generate cache during build

The generated cache is not used in the package. It causes spurious build
failures on the arm64 and ppc64el builds, that are not reproducible
elsewhere.

Gbp-Pq: Name 0002-Do-not-generate-cache-during-build.patch

configure: fix linking on RISC-V ISA

Gbp-Pq: Name 0001-configure-fix-linking-on-RISC-V-ISA.patch

vlc (3.0.21-7) unstable; urgency=medium

  * debian/patches: Fix build with x265 4.1

[dgit import unpatched vlc 3.0.21-7]

Import vlc_3.0.21-7.debian.tar.xz

[dgit import tarball vlc 3.0.21-7 vlc_3.0.21-7.debian.tar.xz]

Import vlc_3.0.21.orig.tar.xz

[dgit import orig vlc_3.0.21.orig.tar.xz]